Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 28, 2025

Modernizes dependency injection across all Angular components and services by replacing constructor-based injection with the inject() function pattern.

Changes

  • 15 components migrated: Layout, pairing (3), planning actions (5), planning additions (3), planning pages (3)
  • 7 services migrated: All plugin services and state management services

Pattern Applied

Before:

constructor(
  private service: ServiceType,
  public dialog: MatDialog
) {}

After:

private service = inject(ServiceType);
public dialog = inject(MatDialog);

constructor() {}

Constructor logic preserved where needed. All visibility modifiers (private/public) maintained.

Files changed: 22 files, 138 insertions(+), 146 deletions(-)

Original prompt

This section details on the original issue you should resolve

<issue_title>Migrate all components from constructor-based DI to inject() pattern</issue_title>
<issue_description>Use this PR as reference for all the parts that needs to be taken into consideration when doing the migration microting/eform-backendconfiguration-plugin#608</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Migrate all components to inject() pattern for dependency injection Migrate components and services from constructor-based DI to inject() pattern Nov 28, 2025
Copilot AI requested a review from renemadsen November 28, 2025 15:33
Copilot finished work on behalf of renemadsen November 28, 2025 15:33
@renemadsen renemadsen marked this pull request as ready for review November 28, 2025 15:43
@renemadsen renemadsen merged commit 9a7e67a into master Nov 28, 2025
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate all components from constructor-based DI to inject() pattern

3 participants